Skip to content

Update to use BLIS 2.0 #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update to use BLIS 2.0 #25

wants to merge 4 commits into from

Conversation

ViralBShah
Copy link
Member

No description provided.

@ViralBShah
Copy link
Member Author

ViralBShah commented Oct 4, 2025 •
edited
Loading

@imciner2 @giordano Any ideas what's happening here?

On my local machine (M2 Mac), I see:

[ Info: blis_jll yields BLIS installation: /Users/viral/.julia/artifacts/043b26a2c0a497f9e28fa1ffa072488cc3e43e58/lib/libblis.4.0.0.dylib.

libblis: frame/base/bli_gks.c (line 257):
libblis: Default MC is non-multiple of MR for one or more datatypes.
libblis: Aborting.

[64319] signal 6: Abort trap: 6
in expression starting at /Users/viral/.julia/dev/BLIS/test/init_test_mmul.jl:23
__pthread_kill at /usr/lib/system/libsystem_kernel.dylib (unknown line)
Allocations: 55326551 (Pool: 55325801; Big: 750); GC: 50
ERROR: Package BLIS errored during testing (received signal: 6)

@ViralBShah ViralBShah changed the title Bump version to 0.3.0 and update to use BLIS 2.0 Update to use BLIS 2.0 Oct 4, 2025
@imciner2
Copy link

imciner2 commented Oct 4, 2025

That's interesting. Can you run with BLIS_ARCH_DEBUG=1 set to see which kernel it is trying to use? That error message apparently means there is a problem with the parameters of the kernel:

	// Verify that cache blocksizes are whole multiples of register blocksizes.
	// Specifically, verify that:
	//   - MC is a whole multiple of MR.

@ViralBShah
Copy link
Member Author

Doesn't seem to yield much more info:

[ Info: blis_jll yields BLIS installation: /Users/viral/.julia/artifacts/043b26a2c0a497f9e28fa1ffa072488cc3e43e58/lib/libblis.4.0.0.dylib.

libblis: frame/base/bli_gks.c (line 257):
libblis: Default MC is non-multiple of MR for one or more datatypes.
libblis: Aborting.

[65180] signal 6: Abort trap: 6
in expression starting at /Users/viral/.julia/dev/BLIS/test/init_test_mmul.jl:23
__pthread_kill at /usr/lib/system/libsystem_kernel.dylib (unknown line)
Allocations: 54541725 (Pool: 54540988; Big: 737); GC: 48
ERROR: Package BLIS errored during testing (received signal: 6)

@ViralBShah ViralBShah requested a review from Copilot October 5, 2025 01:54
Copilot
Copilot AI reviewed Oct 5, 2025
View reviewed changes
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request updates the CI workflow configuration to modernize the GitHub Actions setup and dependencies. The changes focus on updating action versions, improving caching, and refining the test matrix configuration.

  • Updated GitHub Actions to newer versions (checkout@v5, setup-julia@v2, cache@v2, codecov-action@v5)
  • Simplified Julia version testing to focus on version '1' instead of specific versions
  • Added concurrency controls and ARM64 support for macOS

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@giordano giordano giordano left review comments

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ViralBShah @imciner2 @giordano